popupMenu: Fix RemoteMenu items with boolean state action

Stateful actions are expected to pass their state when activated,
but we currently only do this for actions with a string state.

https://bugzilla.gnome.org/show_bug.cgi?id=674932

diff --git a/js/ui/popupMenu.js b/js/ui/popupMenu.js
index 09a9b737dc1e316043756191cb9394475268fda7..b1a06ed9c46acb5a40d78a36e05156039edafc86 100644 (file)
--- a/js/ui/popupMenu.js
+++ b/js/ui/popupMenu.js
@@ -1867,7 +1867,7 @@ const RemoteMenu = new Lang.Class({
                 item = new PopupSwitchMenuItem(label, action.state.get_boolean());
                 action.items.push(item);
                 specificSignalId = item.connect('toggled', Lang.bind(this, function(item) {
-                    this.actionGroup.activate_action(action_id, null);
+                    this.actionGroup.activate_action(action_id, GLib.Variant.new_boolean(item.state));
                 }));
                 break;
             case 's':